Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Finishes #187355064] Users should be able to chat #31

Merged
merged 2 commits into from
Jul 26, 2024
Merged

Conversation

lilika67
Copy link
Collaborator

What Does This PR DO?

This is the feature that will allow users to chat.
It uses WebSocket to allow bidirectional communication between users who are currently online.

Why is this important?

Having travellers interacting with each other on the platform adds value to it as it allows them to asks questions about places, offer recommendations, guidance

Acceptance Criteria

GIVEN the user is online
WHEN they open the chat tab
THEN they should be able to read all of the past chats

GIVEN the user is online
WHEN they open the chat tab
THEN they should be able to post a public chat

GIVENthe user
WHEN they post a chat
THEN it should be able to be echoed to all online users in real time.

ScreenShoot

image

Copy link

netlify bot commented Jul 24, 2024

Deploy Preview for dponesandzereos ready!

Name Link
🔨 Latest commit e58b174
🔍 Latest deploy log https://app.netlify.com/sites/dponesandzereos/deploys/66a39c1780037d0008037836
😎 Deploy Preview https://deploy-preview-31--dponesandzereos.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

netlify bot commented Jul 24, 2024

Deploy Preview for onezerosfrontennd ready!

Name Link
🔨 Latest commit e58b174
🔍 Latest deploy log https://app.netlify.com/sites/onezerosfrontennd/deploys/66a39c1742f8e90009411b3f
😎 Deploy Preview https://deploy-preview-31--onezerosfrontennd.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

netlify bot commented Jul 24, 2024

Deploy Preview for onesandzeroesshop ready!

Name Link
🔨 Latest commit e58b174
🔍 Latest deploy log https://app.netlify.com/sites/onesandzeroesshop/deploys/66a39c1761cb8b0008b07c68
😎 Deploy Preview https://deploy-preview-31--onesandzeroesshop.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Collaborator

@UmuhireJessie UmuhireJessie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@lilika67

  1. Please consider moving the closing icon in the chat modal
Screenshot 2024-07-24 at 16 55 29
  1. Chat are not going through
Screenshot 2024-07-24 at 17 00 24
  1. From the above, please create the chat so that it is overlay over the product lists instead of spacing it out

@lilika67 lilika67 force-pushed the chat branch 3 times, most recently from c372845 to 19ed970 Compare July 25, 2024 17:13
Copy link

netlify bot commented Jul 25, 2024

Deploy Preview for onesandzeros ready!

Name Link
🔨 Latest commit e58b174
🔍 Latest deploy log https://app.netlify.com/sites/onesandzeros/deploys/66a39c17c138b60008ac5605
😎 Deploy Preview https://deploy-preview-31--onesandzeros.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@lilika67 lilika67 force-pushed the chat branch 3 times, most recently from 777934e to 807d7d3 Compare July 26, 2024 11:06
@shema-surge shema-surge self-requested a review July 26, 2024 11:40
[#187355060] Admin Page for viewing users
@UmuhireJessie UmuhireJessie merged commit c4810a0 into develop Jul 26, 2024
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants